Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ref: remote updates #4062

Merged
merged 123 commits into from
Feb 17, 2023
Merged

ref: remote updates #4062

merged 123 commits into from
Feb 17, 2023

Conversation

jorgeorpinel
Copy link
Contributor

@jorgeorpinel jorgeorpinel commented Oct 24, 2022

some updates around the topic in existing docs
add prospective figure titles
and some links from cmd refs
and avoid term "data remote"
and some admons nearby...
and add Project config section to DM/ DV guide
@jorgeorpinel jorgeorpinel added the A: docs Area: user documentation (gatsby-theme-iterative) label Oct 24, 2022
Copy link
Contributor

@dberenbaum dberenbaum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good @jorgeorpinel except for a couple places where the cloud versioning info is not quite correct.

@dberenbaum
Copy link
Contributor

Also, I don't think it closes #4089? I see items there that this doesn't address.

@shcheklein shcheklein temporarily deployed to dvc-org-guide-data-mgmt-msfj9w February 2, 2023 16:33 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-guide-data-mgmt-msfj9w February 2, 2023 16:38 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-guide-data-mgmt-msfj9w February 2, 2023 16:46 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-guide-data-mgmt-msfj9w February 7, 2023 19:44 Inactive
Copy link
Contributor

@dberenbaum dberenbaum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jorgeorpinel!

@dberenbaum
Copy link
Contributor

@jorgeorpinel Can we resolve the conflicts and merge this one?

@shcheklein shcheklein temporarily deployed to dvc-org-guide-data-mgmt-msfj9w February 17, 2023 06:59 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-guide-data-mgmt-msfj9w February 17, 2023 07:04 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-guide-data-mgmt-msfj9w February 17, 2023 07:05 Inactive
certain cloud storage providers). By default, DVC will automatically do so
only if the `url` contains a valid cloud versioning ID. Otherwsie, with this
flat DVC will import the latest version of the file.

- `-h`, `--help` - prints the usage/help message, and exit.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just noticed that this sentence is wrong hah. Should update everywhere (separate PR) to

Suggested change
- `-h`, `--help` - prints the usage/help message, and exit.
- `-h`, `--help` - print the usage/help message and exit.

@jorgeorpinel
Copy link
Contributor Author

@dberenbaum conflicts solved 🙂 PTAL/ merge at your discretion. Thanks

@github-actions
Copy link
Contributor

github-actions bot commented Feb 17, 2023

Link Check Report

There were no links to check!

@shcheklein shcheklein temporarily deployed to dvc-org-guide-data-mgmt-msfj9w February 17, 2023 07:13 Inactive
@shcheklein shcheklein merged commit 108b121 into main Feb 17, 2023
@shcheklein shcheklein deleted the guide/data-mgmt/remote-storage-types branch February 17, 2023 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: docs Area: user documentation (gatsby-theme-iterative) C: guide Content of /doc/user-guide C: ref Content of /doc/*-reference type: enhancement Something is not clear, small updates, improvement suggestions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants